Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cheshire Terminal #455

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Cheshire Terminal #455

wants to merge 17 commits into from
+38,619 −14,064

Conversation

8bitsats
Copy link

Cheshire Terminal
https://zany-cantaloupe-d3d.notion.site/Cheshire-Terminal-142ea810c1f7803f9040c91541e710e5?pvs=73

Low

-->

Background

What does this PR do?

A new Character, Cross Chain, Memetic Warfare, Adds music, and verifies the Cheshire Terminal $grin token launch through Eliza Fork and framework. Also want to establish a foundation for Cheshire Framework integrating metaplex and helius as well as cross chain zero knowledge recursion using on chain ai oracles using metaplex core, zero knowledge recursion, and cross chain communication of assets on Bitcoin, Solana, And Beyond.

What kind of change is this?

Features, Character Add.

This is my first official pull request on any github and I would be honored if it would be included in the Eliza Framework. I have been working on Cross Chain Multi Mo del Ai Agents, on and off chain for about 6 months now, and have specifically trained data sets and fine tunes models for Solana Bitcoin and beyond.

Documentation changes needed?

MemeticWarfare
trading
characters/cheshire
eliza/music

Testing

Where should a reviewer start?

Detailed testing steps

@snobbee
Copy link
Contributor

snobbee commented Nov 20, 2024

@8bitsats please squash all your commits into one with a more relevant commit message.

@jkbrooks
Copy link
Contributor

There are also conflicts that would need to be addressed in the following files:

.env.example
docs/docs/guides/configuration.md
package-lock.json
packages/adapter-sqlite/package.json
packages/agent/src/index.ts
packages/client-discord/src/messages.ts
packages/client-discord/src/voice.ts
packages/client-discord/tsconfig.json
packages/client-telegram/src/messageManager.ts
packages/client-telegram/src/telegramClient.ts
packages/client-twitter/package.json
packages/client-twitter/src/index.ts
packages/client-twitter/src/interactions.ts
packages/client-twitter/src/post.ts
packages/core/src/embedding.ts
packages/core/src/generation.ts
packages/core/src/memory.ts
packages/core/src/runtime.ts
packages/core/tsconfig.json
pnpm-lock.yaml

It seems that some of these changes are specific to a character you want to create @8bitsats , and those would be better served for your own fork of Eliza. There are various forks as you can see in Eliza World , but people display that content in their own hosted fork rather than in the main repo.

Then the last point of feedback is that this is a very large pull request, it would help if you broke down the changes into smaller pull requests that we could review more easily.

And then I would say we actually may want to add this to the guidelines of our contributing docs or some other guidelines cc @monilpat @lalalune

@8bitsats
Copy link
Author

hey brother, this is my first pull request, and I totally sent you my local copy before I cleaned up the build post token launch, give me a few and I'll organize it and clean it up for you. Thanks again for the response!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@8bitsats @snobbee @jkbrooks